[UP][m] Updating the scripts with v2 source of world bank #3
Conversation
There was a problem hiding this comment.
Hey @gradedSystem ! Nice work, but lets remove this README in scripts and merge it into README in root directory, all the contents move to Preparation section as in https://github.com/datasets/currency-codes. Also include how to run make file as in your github actions. Lets make a same structure. Also please remove some unnecessary headings such as Usage of process.py and Tags and committing changes and make it simple and easy to read.
|
@Mikanebu done |
README.md
Outdated
| but please follow [OKFN's curator guide](http://data.okfn.org/doc/core-data-curators) | ||
| to keep things as organized as possible. | ||
|
|
||
| ## Tags and committing changes |
README.md
Outdated
|
|
||
| ## License | ||
|
|
||
| All data is licensed under the Open Data Commons Public Domain Dedication and License. All code is licensed under the MIT/BSD license. | ||
|
|
||
| Note that while no credit is formally required a link back or credit to Rufus Pollock and the Open Knowledge Foundation is much appreciated. | ||
|
|
||
|
|
||
| ## Contributing |
There was a problem hiding this comment.
Also this one, since that url is not valid anymore
README.md
Outdated
| @@ -7,9 +7,40 @@ The repository of the data package of the GINI Index. | |||
| The data that is contained in the `gini-index.csv` file, under `/data` was | |||
| retrieved from [the World Bank](http://data.worldbank.org/indicator/SI.POV.GINI). | |||
|
|
|||
| ## Requirements | |||
There was a problem hiding this comment.
This one should be Preparation section and add some sentence on how to run scripts
README.md
Outdated
| run the script. | ||
|
|
||
| `pip install -r scripts/requirements.txt` | ||
|
|
||
|
|
||
| ## License |
There was a problem hiding this comment.
Before the license, add Automation section as in https://github.com/datasets/gdp
| Run the requirements through `pip` in order to install all required packages to | ||
| run the script. | ||
|
|
||
| `pip install -r scripts/requirements.txt` |
There was a problem hiding this comment.
after this add how to run script for example:
Process is recorded and automated in python script:
scripts/process.py
|
@Mikanebu Yes it is |
|
@gradedSystem Pls, update then accordingly. |
Changes made
archiveanddatafolderscc @anuveyatsu